Content-wise, it looks great to me 💪

Consider getting a Forms expert (like Ronald), to give a code review.

A long overdue comment: I haven't been able to exactly replicate the behavior documented here. All workflows will be executed by default on either submission or approval, regardless of whether one of them failed or sets the record state to rejected. However, once e.g. all submission workflows have executed and one of them sets the record state to rejected, the approval workflows will indeed be skipped (there's no need to return a failed workflow execution status though).

It is possible to add a condition to a workflow to skip execution based on a field value, but that would require 'abusing' a (hidden) field and populating it within another workflow. I wouldn't recommend this though and suggest inheriting the workflow you want to conditionally skip instead to add the conditional logic inside the ExecuteAsync() method.